feat(landing): add Share chip to integration and model pages#5582
Conversation
- add a Share chip (copy link / X / LinkedIn) to integration and
model detail pages, matching the bordered secondary-pill chip
already used for View docs / All {provider} models
- rework ShareButton to render as a Chip everywhere (blog, library,
integrations, models) instead of a bespoke muted-text trigger, and
switch its copy-link state to the shared useCopyToClipboard hook
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
PR SummaryLow Risk Overview
Reviewed by Cursor Bugbot for commit 949539a. Configure here. |
Greptile SummaryThis PR adds sharing controls to landing detail pages. The main changes are:
Confidence Score: 5/5This looks safe to merge.
Important Files Changed
Reviews (1): Last reviewed commit: "feat(landing): add Share chip to integra..." | Re-trigger Greptile |
Summary
ShareButtonto always render as a borderedChip(TRIGGER_BORDER_CLASS), matching the existing "View docs" / "All {provider} models" secondary pills exactly, instead of a bespoke muted-text button — now consistent across blog, library, integration, and model pagesuseCopyToClipboardhook from@sim/emcnType of Change
Testing
Tested manually — ran the landing app locally and confirmed the Share chip renders with byte-identical chip chrome to its sibling chips on
/integrations/[slug],/models/[provider]/[model], and existing/blog/[slug]pages (verified via raw SSR HTML class comparison). Typecheck and lint pass with no new errors/warnings.Checklist